Conversation
…에 들어온 부분 수정, certify SecurityConfiguration에 추가,userservice에서 CertifyService를 사용해서 검증하는걸로 수정
DaveLee-b
requested review from
jeemin65-pixel,
jong15325,
jungyoyang and
munyeong0103
jong15325
approved these changes
Jan 7, 2026
Contributor
jong15325
left a comment
jong15325
left a comment
There was a problem hiding this comment.
전체적으로 빠르게 확인했습니다 동석님이라 믿습니다!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝요약(Summary)
이슈 번호 : #131
🔨변경 사항(Changes)
회원가입시 이메일을 검증하는 기능입니다
😉리뷰 요구사항
미처 생각치 못한 허점이나 기능 로직상에 이상이 있다면 말씀해주시고 해당 기능을 개발을 했지만 테스트는 현재 시간상 그리고 혹시 모를 사고를 미연에 방지하기위해 로컬에서 진행했습니다. 최종발표때 해당 내용을 적용해서 시연영상을 찍을지,언급하고 넘어갈지 의견 주시면 감사하겠습니다.